Skip to content

HIVE-30017: Bump postgres driver version in docker README to 42.7.12 - #6769

Merged
ayushtkn merged 2 commits into
apache:masterfrom
Akanksha-kedia:bump-docker-readme-postgres-driver-42.7.12
Sep 10, 2026
Merged

ayushtkn merged 2 commits into
apache:masterfrom
Akanksha-kedia:bump-docker-readme-postgres-driver-42.7.12

Conversation

@Akanksha-kedia

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Update the example commands in packaging/src/docker/README.md that reference the postgres JDBC driver, bumping the version from 42.7.3 to 42.7.12.

Why are the changes needed?

Keep the documented driver version current.

Does this PR introduce any user-facing change?

No, docs only.

Is the change a dependency upgrade?

No, this only updates a documentation example; no build dependency is changed.

@Akanksha-kedia Akanksha-kedia changed the title Bump postgres driver version in docker README to 42.7.12 [HIVE-30017]Bump postgres driver version in docker README to 42.7.12 Sep 9, 2026
@Akanksha-kedia Akanksha-kedia changed the title [HIVE-30017]Bump postgres driver version in docker README to 42.7.12 HIVE-30017: Bump postgres driver version in docker README to 42.7.12 Sep 9, 2026
@Akanksha-kedia

Copy link
Copy Markdown
Contributor Author

cc @abstractdog

@Akanksha-kedia

Copy link
Copy Markdown
Contributor Author

cc @ayushtkn @uros-b

Update example commands referencing the postgres JDBC driver from
42.7.3 to 42.7.12.
… 42.7.12

Same stale example version left over in standalone-metastore's docker
README.
@Akanksha-kedia
Akanksha-kedia force-pushed the bump-docker-readme-postgres-driver-42.7.12 branch from e19d882 to 15391cd Compare September 10, 2026 06:05
@sonarqubecloud

Copy link
Copy Markdown

@ayushtkn ayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well this is not that important, as long as the version mentioned in the README works with Hive, but I don't mind keeping it updated.

Can you try the changed commands once as well & see they run fine.

Thanx @Akanksha-kedia for the update, @tanishq-chugh can you give a shot as well

@tanishq-chugh tanishq-chugh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1
Tried out the updated commands and it works fine for me:
Image

Image

(network and path changes in commands here are as per my setup).

@Akanksha-kedia

Copy link
Copy Markdown
Contributor Author

@ayushtkn CI is green here (rebased on latest master) — could you take a look and merge if it looks good?

@ayushtkn
ayushtkn merged commit 2be30b7 into apache:master Sep 10, 2026
7 checks passed
@ayushtkn

Copy link
Copy Markdown
Member

Thanx @Akanksha-kedia and @tanishq-chugh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants